Skip to content

[PWGLF] remove daughter indices - #17224

Merged
mpuccio merged 1 commit into
AliceO2Group:masterfrom
mpuccio:genericHe3body
Jul 27, 2026
Merged

[PWGLF] remove daughter indices#17224
mpuccio merged 1 commit into
AliceO2Group:masterfrom
mpuccio:genericHe3body

Conversation

@mpuccio

@mpuccio mpuccio commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions

github-actions Bot commented Jul 27, 2026

Copy link
Copy Markdown

O2 linter results: ❌ 35 errors, ⚠️ 4 warnings, 🔕 2 disabled

@github-actions github-actions Bot changed the title PWGLF: remove daughter indices [PWGLF] remove daughter indices Jul 27, 2026
@mpuccio
mpuccio enabled auto-merge (squash) July 27, 2026 14:18
alibuild
alibuild approved these changes Jul 27, 2026
View reviewed changes

@alibuild alibuild left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approving on behalf of @mpuccio.

@alibuild

alibuild commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 95fe2de at 2026-07-27 16:38:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 14 errors and 0 warnings.'
Found 14 errors and 0 warnings.
++ [[ 14 -gt 0 ]]
++ cat /sw/BUILD/9bb503ce465127fff2723393c1e01c2351728730/O2Physics-code-check/errors.txt
PWGLF/DataModel/Vtx3BodyTables.h:112:42: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/DataModel/Vtx3BodyTables.h:113:38: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/DataModel/Vtx3BodyTables.h:114:46: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/DataModel/Vtx3BodyTables.h:115:42: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/nucleiThreeBodyBuilder.cxx:49:1: error: included header limits is not used directly [misc-include-cleaner,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/nucleiThreeBodyBuilder.cxx:93:20: error: no header providing "PDG_t" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/nucleiThreeBodyBuilder.cxx:98:8: error: constructor does not initialize these fields: ccdb [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/nucleiThreeBodyBuilder.cxx:99:3: error: no header providing "o2::framework::Produces" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/nucleiThreeBodyBuilder.cxx:102:3: error: no header providing "o2::framework::Service" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/nucleiThreeBodyBuilder.cxx:103:46: error: no header providing "o2::framework::OutputObjHandlingPolicy" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/nucleiThreeBodyBuilder.cxx:105:3: error: no header providing "o2::framework::OutputObj" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/nucleiThreeBodyBuilder.cxx:129:18: error: no header providing "o2::framework::LabeledArray" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/nucleiThreeBodyBuilder.cxx:262:10: error: no header providing "size_t" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/nucleiThreeBodyBuilder.cxx:266:70: error: no header providing "o2::framework::HistType" is directly included [misc-include-cleaner,-warnings-as-errors]
++ [[ 0 -gt 0 ]]
++ [[ 14 -gt 0 ]]
++ exit 1
--

Full log here.

@mpuccio
mpuccio merged commit c33f338 into AliceO2Group:master Jul 27, 2026
12 of 15 checks passed
@mpuccio
mpuccio deleted the genericHe3body branch July 27, 2026 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alibuild alibuild alibuild approved these changes
@omvazque omvazque Awaiting requested review from omvazque omvazque is a code owner
@skundu692 skundu692 Awaiting requested review from skundu692 skundu692 is a code owner
@fmazzasc fmazzasc Awaiting requested review from fmazzasc fmazzasc is a code owner
@maciacco maciacco Awaiting requested review from maciacco maciacco is a code owner
@HorstMa HorstMa Awaiting requested review from HorstMa HorstMa is a code owner
@gbencedi gbencedi Awaiting requested review from gbencedi gbencedi is a code owner
@abmodak abmodak Awaiting requested review from abmodak abmodak is a code owner
@dmallick2 dmallick2 Awaiting requested review from dmallick2 dmallick2 is a code owner
@smaff92 smaff92 Awaiting requested review from smaff92 smaff92 is a code owner
@ercolessi ercolessi Awaiting requested review from ercolessi ercolessi is a code owner
@romainschotter romainschotter Awaiting requested review from romainschotter romainschotter is a code owner
@prottayCMT prottayCMT Awaiting requested review from prottayCMT prottayCMT is a code owner
@lhusova lhusova Awaiting requested review from lhusova lhusova is a code owner

Assignees

No one assigned

Labels

datamodel pwglf

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mpuccio @alibuild